feat (UI): Mobile: Remove titles on Hot page - #2343
Conversation
Greptile SummaryThe PR hides the Hot page's section headings below the Confidence Score: 4/5The PR should be fixed before merging because it removes the only active-users section label from the mobile user-not-found profile view. Responsive visibility was placed inside a reusable organism, so a caller outside the Hot page loses its heading without receiving the Hot page's replacement tab label. Files Needing Attention: src/components/organisms/HotActiveUsers/HotActiveUsers.tsx
|
| Filename | Overview |
|---|---|
| src/components/organisms/HotActiveUsers/HotActiveUsers.tsx | Hides the Active users heading below lg, unintentionally removing the label when this organism is reused by the mobile user-not-found profile view. |
| src/components/organisms/HotTagsCardsSection/HotTagsCardsSection.tsx | Consistently hides the Hot tags heading below lg across success, loading, and error states as intended for the mobile Hot page. |
| src/components/templates/Feed/Hot/Hot.tsx | Hides the Trending posts heading below lg, where the mobile tab navigation already identifies the active section. |
Reviews (1): Last reviewed commit: "remove titles on hot page for mobile U.I" | Re-trigger Greptile
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
| return ( | ||
| <Container overrideDefaults className={cn('flex w-full flex-col gap-2', className)} data-testid="hot-active-users"> | ||
| <Heading level={5} size="lg" className="font-light text-muted-foreground"> | ||
| <Heading level={5} size="lg" className="hidden font-light text-muted-foreground lg:block"> |
There was a problem hiding this comment.
When a nonexistent profile is opened below the lg breakpoint, ProfileUserNotFoundDiscoveryView renders this reusable component without the Hot page's tab navigation, so hiding the heading leaves the active-users list unlabeled.
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
1 participant
Closes #2271
Removed titles on Hot page for mobile and smaller screens.
Screenrecording
Screen.Recording.2026-08-13.142901.mp4